-
Notifications
You must be signed in to change notification settings - Fork 1.8k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[Backport 2.x] Reduce amount of docs ingested for SegRep ITs #11237
Conversation
This change reduces the amount of docs we ingest inside SegmentReplicationIT. These tests were often ingesting 1-200 docs where it was not required. Many only required a few so that segments are created. This speeds up tests when run with remote store through SegmentReplicationUsingRemoteStore. Also add Testlogging annotation on the replication package for known flaky tests. Signed-off-by: Marc Handalian <handalm@amazon.com> (cherry picked from commit e3bd0c3) Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Compatibility status:Checks if related components are compatible with change e4a051e Incompatible componentsIncompatible components: [https://github.com/opensearch-project/performance-analyzer.git] Skipped componentsCompatible componentsCompatible components: [https://github.com/opensearch-project/security-analytics.git, https://github.com/opensearch-project/custom-codecs.git, https://github.com/opensearch-project/asynchronous-search.git, https://github.com/opensearch-project/observability.git, https://github.com/opensearch-project/notifications.git, https://github.com/opensearch-project/reporting.git, https://github.com/opensearch-project/job-scheduler.git, https://github.com/opensearch-project/common-utils.git, https://github.com/opensearch-project/neural-search.git, https://github.com/opensearch-project/anomaly-detection.git, https://github.com/opensearch-project/ml-commons.git, https://github.com/opensearch-project/index-management.git, https://github.com/opensearch-project/geospatial.git, https://github.com/opensearch-project/sql.git, https://github.com/opensearch-project/security.git, https://github.com/opensearch-project/alerting.git, https://github.com/opensearch-project/cross-cluster-replication.git, https://github.com/opensearch-project/k-nn.git, https://github.com/opensearch-project/performance-analyzer-rca.git] |
❌ Gradle check result for e4a051e: FAILURE Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change? |
|
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## 2.x #11237 +/- ##
============================================
+ Coverage 70.92% 70.96% +0.04%
- Complexity 58958 59012 +54
============================================
Files 4867 4867
Lines 278457 278457
Branches 40810 40810
============================================
+ Hits 197485 197607 +122
+ Misses 64142 64116 -26
+ Partials 16830 16734 -96 ☔ View full report in Codecov by Sentry. |
Backport e3bd0c3 from #10584.